Conversation
…ts guides Curated guidance the code cannot state about itself — which transport scheme, which boundary treatment, how to make a hard solve converge — was split between the AI skills in .claude/skills and the rulings in CLAUDE.md, two paths with no index and no reader outside an assistant. Each guide is now one MyST page under docs/developer/guides with front matter naming the families it applies to; the seven skills are symlinks to those pages, so a skill cannot drift from its guide. Two guides are new: transport schemes, drafted from what the tests established this month with the rulings still open marked as such, and the boundary-condition rulings, gathered from CLAUDE.md and the issues. uw.capabilities() reads the front matter and lists a guide beside its family; the class-level view does the same, so uw.systems.Stokes.view() ends with the guides that apply to Stokes; the server serves them with uw_guides and uw_guide. The developer index carries the table. The adversarial review contract gains the rule, and CLAUDE.md the pointer: a change to a family is reviewed against every guide that names it, in the same change. test_0030 fails on a copied skill, a guide without front matter, or a family no class carries. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Na7qBenCp67rDTZhFGTh5V
Member
Author
|
Merged into #790 with the rest of the stack after review; nothing lost, the commits are there. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Curated guidance the code cannot state about itself was split between
.claude/skillsand the rulings inCLAUDE.md. Each guide is now one MyST page underdocs/developer/guides/with front matter (name,description,families,kind); the seven skills are symlinks to those pages, so there is one source. Two guides are new:transport-schemes(drafted from this month's tests, with the open rulings marked) andboundary-condition-rulings(gathered fromCLAUDE.mdand the issues).The library reads the front matter:
uw.capabilities()lists a guide beside its family,uw.systems.Stokes.view()ends with the guides that apply to Stokes, and the server serves them (uw_guides,uw_guide). The developer index carries a "Capability guides" table.The review contract: a change to a solver family, constitutive model, history scheme or boundary mechanism is reviewed against every guide that names it, in the same change.
docs/developer/guides/adversarial-review.mdandCLAUDE.mdsay so;test_0030_capability_guides.pyfails on a copied skill, a guide without front matter, or a family name no class carries.Stacked on #802. The two new guides need Louis's rulings on the points marked open.
🤖 Generated with Claude Code
https://claude.ai/code/session_01Na7qBenCp67rDTZhFGTh5V